Skip to content

Fix include/lib path ordering so wolfSSL paths take precedence#372

Merged
padelsbach merged 1 commit intowolfSSL:masterfrom
ColtonWilley:wp_fix_include_order
Mar 17, 2026
Merged

Fix include/lib path ordering so wolfSSL paths take precedence#372
padelsbach merged 1 commit intowolfSSL:masterfrom
ColtonWilley:wp_fix_include_order

Conversation

@ColtonWilley
Copy link
Contributor

When --with-openssl= contains stale wolfSSL headers, the compiler picked them up because OpenSSL's -I path appeared before wolfSSL's in CFLAGS. Swap the wolfSSL and OpenSSL detection blocks so wolfSSL's -I/-L paths are added first and take precedence. Also move AC_CHECK_HEADERS for openssl/core.h after OpenSSL paths are set so it can actually find the header.

Fixes ZD 21214

When --with-openssl=<prefix> contains stale wolfSSL headers, the compiler
picked them up because OpenSSL's -I path appeared before wolfSSL's in
CFLAGS. Swap the wolfSSL and OpenSSL detection blocks so wolfSSL's -I/-L
paths are added first and take precedence. Also move AC_CHECK_HEADERS for
openssl/core.h after OpenSSL paths are set so it can actually find the
header.
@ColtonWilley
Copy link
Contributor Author

ColtonWilley commented Mar 17, 2026

jenkins retest this please

@ColtonWilley ColtonWilley requested a review from padelsbach March 17, 2026 07:24
@padelsbach padelsbach merged commit 144ead6 into wolfSSL:master Mar 17, 2026
482 of 572 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants